Revert "Cargo templating for `new` and `init`"

This reverts commit 875a8aba7916b63c3c8464008a271f6082e23779.

Revert "Upgrade handlebars to 0.25."

This reverts commit 0aad658f09590396b4c7fe6ce93eae322354fd06.

Revert "Fix for #3722"

This reverts commit 7e80c684cad3249923b05ecbe02c009b111d9b9f.

Revert "Use time crate directly to get the year"

This reverts commit 47221e98189daa34e47934d44223db12db84c7b9.

Revert "Add year to project template variables"

This reverts commit 69ffd99f4261b10414b882eed638f041e42d8f73.

Auto merge of #3869 - matklad:test-isolation, r=alexcrichton

Don't read ~/.cargo/config in tests

Closes #3863

r? @alexcrichton

Don't read ~/.cargo/config in tests

Closes #3863

Auto merge of #3865 - matklad:doc-all-the-things, r=alexcrichton

Beef up docs about integrating Cargo with other stuff

Add info about `--message-format=json`, `--dep-info` and custom cargo subcommands (mostly copy-pasted from the wiki :) ).

cc @carols10cents

r? @alexcrichton

Add external-tools.md to the Makefile

Auto merge of #3862 - matklad:crate-types, r=alexcrichton

Fix crate_types in serialized targets

closes #3840

Beef up docs about integrating Cargo with other stuff

Fix crate_types in serialized targets

Auto merge of #3857 - antonlarin:rebuild-on-env-change, r=alexcrichton

Include package props with corresponding env vars into target metadata

Previously, when changing package properties with corresponding environment variables (such as authors, which has CARGO_PKG_AUTHORS), it didn't invalidate the build, even though there could have been a dependency on such variables in the source code.

This commit includes 3 such properties: authors list, description and homepage in the target metadata.

I've added a test only for description change, can add more if necessary.
Fixes #3696.

Auto merge of #3837 - alexcrichton:workspace-exlucde, r=brson

Add a workspace.exclude key

This commit adds a new key to the `Cargo.toml` manifest, `workspace.exclude`.
This new key is a list of strings which is an array of directories that are
excluded from the workspace explicitly. This is intended for use cases such as
vendoring where path dependencies into a vendored directory don't want to pull
in the workspace dependencies.

There's a number of use cases mentioned on #3192 which I believe should all be
covered with this strategy. At a bare minimum it should suffice to `exclude`
every directory and then just explicitly whitelist crates through `members`
through inclusion, and that should give precise control over the structure of a
workspace.

Closes #3192

Include package props with env vars into target metadata

Previously, when changing package properties with corresponding
environment variables (such as authors, which has CARGO_PKG_AUTHORS),
it didn't invalidate the build, even though there could have been
a dependency on such variables in the source code.

This commit includes such properties (there are 3 of them in total:
authors, description and homepage) in the target metadata.

Fixes #3696.

Auto merge of #3858 - alexcrichton:fix-ci, r=alexcrichton

Attempt to fix CI

Travis looks to have switched to rustup.rs

Attempt to fix CI

Travis looks to have switched to rustup.rs

Auto merge of #3841 - matklad:encourage-explicit-version, r=alexcrichton

Encourage tools writers to explicitly pin metadata version

We do support versioning of metadata, but let's encourage tool's writers to actually use it.

They might not realize that this flag exists at all, or they can be too lazy (like myself :( ) to use it.

We can also make this flag mandatory, but I think that's a little bit to far.

Auto merge of #3848 - tee-too:fix-2529, r=alexcrichton

Report the name of the test that failed (fix #2529)

Fix #2529

Auto merge of #3849 - alexcrichton:fix-stack-overflow, r=matklad

Detect cyclic dependencies through [replace]

Previously this'd cause a stack overflow in Cargo later in the compilation
graph, but this is intended to get caught during resolution.

Closes #3831

Detect cyclic dependencies through [replace]

Previously this'd cause a stack overflow in Cargo later in the compilation
graph, but this is intended to get caught during resolution.

Closes #3831

Report the name of the test that failed (fix #2529)

Auto merge of #3843 - alexcrichton:fix-override-default-features, r=matklad

Fix overriding mixing with default features

Previously Cargo had a bug where if a crate *without* a default feature was
overridden with one that did indeed have a default feature then the default may
not end up getting activated by accident. The fix was to avoid returning too
quickly hen activating dependencies until after we've inspected and learned
about replacements.

Closes #3812

Fix overriding mixing with default features

Previously Cargo had a bug where if a crate *without* a default feature was
overridden with one that did indeed have a default feature then the default may
not end up getting activated by accident. The fix was to avoid returning too
quickly hen activating dependencies until after we've inspected and learned
about replacements.

Closes #3812

Encourage tools writers to explicitly pin metadata version

Auto merge of #3839 - tee-too:fix-3828, r=matklad

Remove --cap-lints feature detection (fix issue #3828)

Remove --cap-lints feature detection (fix issue #3828)

Add a workspace.exclude key

This commit adds a new key to the `Cargo.toml` manifest, `workspace.exclude`.
This new key is a list of strings which is an array of directories that are
excluded from the workspace explicitly. This is intended for use cases such as
vendoring where path dependencies into a vendored directory don't want to pull
in the workspace dependencies.

There's a number of use cases mentioned on #3192 which I believe should all be
covered with this strategy. At a bare minimum it should suffice to `exclude`
every directory and then just explicitly whitelist crates through `members`
through inclusion, and that should give precise control over the structure of a
workspace.

Closes #3192

Auto merge of #3836 - alexcrichton:bump, r=alexcrichton

Bump to 0.19.0

Bump to 0.19.0

Auto merge of #3827 - alexcrichton:warnings-v2, r=matklad

Cap lints for upstream deps with `-vv`

Previously with `-vv` Cargo didn't pass `--cap-lints` at all, but with upstream
dependencies we still want to pass at least `--cap-lints warn` to make sure that
they're all still compiling.

Closes #3823

Cap lints for upstream deps with `-vv`

Previously with `-vv` Cargo didn't pass `--cap-lints` at all, but with upstream
dependencies we still want to pass at least `--cap-lints warn` to make sure that
they're all still compiling.

Closes #3823

Auto merge of #3818 - jmatraszek:3814-run_proper_binary_main_rs, r=alexcrichton

Fixes #3814

Auto merge of #3820 - alexcrichton:remove-build-date, r=alexcrichton

Remove build date from version

This was done by rustc awhile ago anyway and the rustc build system isn't
exporting it.

Tweak travis targets and dates

Just trying to get a PR to land...

Remove build date from version

This was done by rustc awhile ago anyway and the rustc build system isn't
exporting it.

#3814: Fallback to src/bin/main.rs for bins

Auto merge of #3807 - nerdrew:nerdrew/_cargo, r=alexcrichton

support custom cargo-* command zsh tab completion

Auto merge of #3794 - alexcrichton:better-errors, r=matklad

Improve TOML decoding error messages

Unfortunately while `#[serde(untagged)]` is precisely what we want in terms of
semantics it leaves a little to be desired in terms of error messages. This
commit updates to remove the usage of that attribute in favor of implementing
`Deserialize` directly, which is quite simple in these few cases.

Closes #3790

Improve TOML decoding error messages

Unfortunately while `#[serde(untagged)]` is precisely what we want in terms of
semantics it leaves a little to be desired in terms of error messages. This
commit updates to remove the usage of that attribute in favor of implementing
`Deserialize` directly, which is quite simple in these few cases.

Closes #3790

support custom cargo-* command zsh tab completion

Auto merge of #3789 - vojtechkral:cargo_env, r=alexcrichton

Tell subprocesses the path to self in an env variable #3778

I'm just setting the env var on the process itself, letting subprocesses inherit that, as it's easier than setting for each subprocess individually. I'm not entirely sure this is the right spot though.

Also, I should probably document this somewhere - what would be the best place?

Tell subprocesses the path to self in an env variable #3778

Auto merge of #3369 - joshtriplett:cargo-install-only-required-dependencies, r=alexcrichton

cargo fails if it can't find optional dependencies, even if corresponding feature not enabled

I have a directory registry containing all the crate sources needed to build an application crate (for instance, ripgrep), and a `$CARGO_HOME/config` file that looks like this:

```toml
[source.crates-io]
replace-with = "dh-cargo-registry"

[source.dh-cargo-registry]
directory = "/usr/share/cargo/registry/"
```

When I attempt to build ripgrep via "cargo install ripgrep" from that directory registry, I get this error:

```
error: failed to compile `ripgrep v0.3.1`, intermediate artifacts can be found at `/tmp/cargo-install.rmKApOw9BwAL`

Caused by:
  no matching package named `simd` found (required by `bytecount`)
location searched: registry https://github.com/rust-lang/crates.io-index
version required: ^0.1.1
```

The directory registry indeed does not contain "simd"; however, bytecount doesn't require simd.  It has an optional dependency on simd, and nothing enables the feature that requires that dependency.

Placing the simd crate sources into the directory registry allows ripgrep to build; the resulting build does not actually build the simd crate.

I can reproduce this by just trying to build the "bytecount" crate directly, using the same `$CARGO_HOME`:

```
error: no matching package named `simd` found (required by `bytecount`)
location searched: registry https://github.com/rust-lang/crates.io-index
version required: = 0.1.1
```
(Incidentally, that "version required" seems wrong: bytecount has an optional dependency on simd `^0.1.1`, not `=0.1.1`.)

However, this doesn't seem consistent with other crates in the same dependency tree.  For instance, ripgrep also depends on clap, and clap has an optional dependency on yaml-rust, yet cargo does not complain about the missing yaml-rust.

I'd *guess* that the difference occurs because ripgrep has an optional feature `simd-accel` that depends on `bytecount/simd-accel`, so cargo wants to compute what packages it needs for that case too, even when building without that feature. (Similar to #3233.)

However, this makes it impossible to build a package while installing only the packaged dependencies for the enabled features.  Could `cargo install` ignore any dependencies not actually required by the enabled feature?  (That behavior would make no sense for "cargo build", which builds a Cargo.lock file that should remain consistent regardless of enabled features, but it makes sense for "cargo install cratename", which doesn't build a Cargo.lock file.)

Auto merge of #3795 - jryans:template-year, r=alexcrichton

Add year to project template variables

This adds the current year as a `year` variable for project templates. Some license files / headers include the year, so this should make it easier to include those in a template.

In "cargo install" directly from registry, don't require optional dependencies

When building with a directory registry that contains only the subset of
crates required to build an application crate, cargo fails if that
subset doesn't include optional dependencies pulled in for every
possible feature of the root crate, even when the install doesn't enable
those features.  This prevents Linux distributions from building with
a minimal set of dependencies (omitting, for instance, packages for
unstable/nightly features).

Introduce a new workspace flag "require_optional_deps", disabled for
install and enabled for everything else.  Skip the initial
Method::Everything resolve in this case, and modify
resolve_with_previous to support running a Method::Required resolve
without a previous resolution.

This also skips adding path overrides, as those won't make sense (and
won't work) for an install directly from a registry.

Introduce a set of tests for "cargo install" directly from a directory
registry.

Use time crate directly to get the year

Auto merge of #3799 - integer32llc:restore-generated-tests-module, r=alexcrichton

Restore the generated tests module created by `cargo new`

Appears to have been removed unintentionally in #3004.

Was just working on the book, ran `cargo new adder` with cargo-0.18.0-nightly (6f1b860 2017-02-11), and got this in `src/lib.rs`:

```rust
#[test]
fn it_works() {
}
```

when I expected to get this:

```rust
#[cfg(test)]
mod tests {
    #[test]
    fn it_works() {
    }
}
```

It looks like this was changed as part of #3004 ([removed](https://github.com/rust-lang/cargo/commit/875a8aba7916b63c3c8464008a271f6082e23779#diff-149dd4362a3b0dc13b113762713119dfL477), [added](https://github.com/rust-lang/cargo/commit/875a8aba7916b63c3c8464008a271f6082e23779#diff-149dd4362a3b0dc13b113762713119dfR678)), I'm assuming unintentionally?

The regression has not yet hit the beta channel; `cargo-0.17.0-nightly (0bb8047 2017-02-06)` generates the src/lib.rs I expect.

Restore the generated tests module created by `cargo new`

Appears to have been removed unintentionally in #3004.

Add year to project template variables

Auto merge of #3791 - sunng87:port-handlebars-to-serde, r=alexcrichton

Use serde type system for handlebars

This will help cargo to drop rustc_serialize as dependency (#3682). Handlebars actually supports using serde_json as its type system instead of rustc_serialize. And I'm planning to drop rustc_serialize in future releases.

Use serde type system for handlebars

This will help cargo to drop rustc_serialize as dependency

Auto merge of #3782 - alexcrichton:more-proc-macro, r=matklad

Accept proc_macro in addition to proc-macro

More historical behavior...

Auto merge of #3786 - joshtriplett:directory-registry-no-url, r=alexcrichton

tests/directory: Remove unnecessary dummy registry URL for crates.io

Older versions of cargo required this; current versions do not.  All of
the tests still pass with it removed.

Auto merge of #3785 - joshtriplett:insulate-tests-from-user-env, r=alexcrichton

tests: Insulate from user git environment

Several tests in "cargo test" would fail if the user had any of the Git
environment variables set for name or email address, because those
environment variables would override the tested configuration.  Filter
out those environment variables.

tests: Insulate from user git environment

Several tests in "cargo test" would fail if the user had any of the Git
environment variables set for name or email address, because those
environment variables would override the tested configuration.  Filter
out those environment variables.

tests/directory: Remove unnecessary dummy registry URL for crates.io

Older versions of cargo required this; current versions do not.  All of
the tests still pass with it removed.

Auto merge of #3767 - ehiggs:fix-init-bin-lib-section, r=alexcrichton

Fix for #3722

When using init, add [[bin]] and [lib] section where appropriate.

Accept proc_macro in addition to proc-macro

More historical behavior...

Auto merge of #3779 - amotz:appveyor-url, r=alexcrichton

Fix AppVeyor badge URL

Auto merge of #3721 - alexcrichton:dupe-doctest, r=brson

Fix deps with `cargo test --all` and doctests

This commit fixes `cargo test --all` with the way we ship libraries to `rustdoc
--test`. I'm... not entirely sure what the previous incarnation was doing but
the current organization is to interpret `compilation.libraries` as a mapping
from a package to the list of crates it needs to link to test.

This updates the support in `cargo_rustc/mod.rs` to create the map appropriately
and tweaks the loop in `cargo_test.rs` as well.

Closes rust-lang/rust#39879

Fix deps with `cargo test --all` and doctests

This commit fixes `cargo test --all` with the way we ship libraries to `rustdoc
--test`. I'm... not entirely sure what the previous incarnation was doing but
the current organization is to interpret `compilation.libraries` as a mapping
from a package to the list of crates it needs to link to test.

This updates the support in `cargo_rustc/mod.rs` to create the map appropriately
and tweaks the loop in `cargo_test.rs` as well.

Closes rust-lang/rust#39879

Fix AppVeyor badge URL

Auto merge of #3775 - alexcrichton:version-info, r=alexcrichton

Prefer CARGO_PKG_* in version info

These are always set by Cargo and avoids rustbuild-building-cargo to need to
parse semver

Auto merge of #3699 - alexcrichton:sslop, r=brson

Add an option to ignore SSL revocation checking

This is apparently required in some Windows setups to get past SSL context
creation in schannel.

Auto merge of #3776 - alexcrichton:underscores, r=matklad

Accept more underscore variants in configuration

Preserving more historical behavior of the old TOML parser.

Accept more underscore variants in configuration

Preserving more historical behavior of the old TOML parser.

Prefer CARGO_PKG_* in version info

These are always set by Cargo and avoids rustbuild-building-cargo to need to
parse semver

Auto merge of #3771 - alexcrichton:read-more, r=matklad

Accept `default_features` for `default-features`

This was accepted by this historical TOML parser, so we'll need to preserve this
ability.

Closes #3768

Auto merge of #3770 - jbendig:issue_3762, r=alexcrichton

Fixes #3762

Fixes issue #3762. `cargo test` and `cargo bench` should no longer compile binaries with required features that have not been selected.

Accept `default_features` for `default-features`

This was accepted by this historical TOML parser, so we'll need to preserve this
ability.

Closes #3768

Remove tab characters.

:bowing_man:

Fixed compile failing because Resolve::features() no longer returns an Option.

Fixed `cargo test` and `cargo bench` compiling binaries with required features that have not been selected.

Satisfy check-style.sh

Fix for #3722

When using init, add [[bin]] and [lib] section where appropriate.

Auto merge of #3760 - sanmai-NL:Cargo_new_gitignore_rustfmt_.rs.bk, r=alexcrichton

Add `src/**/*.rs.bk` to VCS ignore file

See #2409, specifically https://github.com/rust-lang/cargo/pull/2409#issuecomment-281707650.

Also limit `target` exclusion to `target/` (dirs) as expected.

Auto merge of #3752 - matklad:always-artifacts, r=alexcrichton

Always produce artifact messages

This changes `artifact` messages in several ways:

* They are produced even for fresh builds

* They used the path after hard linking (@jsgf talked about it in the end of https://github.com/rust-lang/cargo/pull/3319#issuecomment-263975431)

* Don't produce filenames if the compiler has not actually produced the binaries (`-Z-no-trans`).

Exclude `*.rs.bk` files more liberally

This will cover them in different places as well, considering the default
[project layout](https://doc.crates.io/guide.html#project-layout).

Produce artifacts JSON messages even for fresh builds

Auto merge of #3742 - matklad:sets-are-monoid, r=alexcrichton

Simplify feature-handling code

A neat (imo :) ) hack to use an empty `&HashSet` instead of `Option<&HashSet>`.

Add `src/**/*.rs.bk` to VCS ignore file

Limit `target` exclusion to `target/` dirs.

Auto merge of #3758 - mbrubeck:docs, r=alexcrichton

Document that OUT_DIR is set during compilation for packages with build scripts

None

Document that OUT_DIR is set during compilation for packages with build scripts

Simplify feature-handling code

Add an option to ignore SSL revocation checking

This is apparently required in some Windows setups to get past SSL context
creation in schannel.

Auto merge of #3682 - alexcrichton:serde, r=brson

Migrate from rustc-serialize to Serde

This commit migrates Cargo as much as possible from rustc-serialize to
Serde. This not only provides an excellent testing ground for the toml
0.3 release but it also is a big boost to the speed of parsing the JSON
bits of the registry.

This doesn't completely excise the dependency just yet as docopt still
requires it along with handlebars. I'm sure though that in time those
crates will migrate to serde!

Migrate from rustc-serialize to Serde

This commit migrates Cargo as much as possible from rustc-serialize to
Serde. This not only provides an excellent testing ground for the toml
0.3 release but it also is a big boost to the speed of parsing the JSON
bits of the registry.

This doesn't completely excise the dependency just yet as docopt still
requires it along with handlebars. I'm sure though that in time those
crates will migrate to serde!

Auto merge of #3733 - llogiq:clippy, r=alexcrichton

some clippy-suggested improvements

This fixes a number of [clippy](https://github.com/Manishearth/rust-clippy) warnings. It's mostly about readability, though a few changes could affect performance (though probably not measurably).

I've left out things to fix later; I thought I'd just push the first batch to see if you like it.

Auto merge of #3743 - jbendig:required_features_doc, r=alexcrichton

Improved required-features docs

This should make it more clear about what happens when not all of the required features are selected.

Auto merge of #3738 - hiseni:master, r=alexcrichton

add help email mention to the policies page

Update policies page to include help email address mention as described here: rust-lang/crates.io#522 :slightly_smiling_face:

Auto merge of #3735 - king6cong:clean, r=alexcrichton

make code more readable

Auto merge of #3734 - king6cong:typo, r=alexcrichton

typo fix

Auto merge of #3731 - alexcrichton:check-all, r=matklad

Add `cargo check --all`

This'll check all `build` targets for all packages in a workspace

Changed wording for required-features docs to be more clear that missing features means the target is skipped.

Auto merge of #3708 - matklad:assert-tests, r=alexcrichton

Assert that we don't build test project twice.

Discussed in https://github.com/rust-lang/cargo/pull/3691#discussion_r100878827.

I've modify the offending tests to be more explicit about recreating projects.

Auto merge of #3737 - jbendig:issue_3727, r=alexcrichton

Fixed required-features not working with dependencies' features.

This should fix issue #3727.

@sgrif suggested making the documentation more clear as well. Should I submit a separate pull request or add a commit for that?

add help email mention to the policies page

improved formatting for bonus points

Fixed some lines being over 100 characters.